Skip to content

docs: fix useLinkClickHandler defaultShouldRevalidate default description#15322

Merged
brophdawg11 merged 1 commit into
remix-run:mainfrom
dfedoryshchev:docs/uselinkclickhandler-revalidate-default
Jul 16, 2026
Merged

docs: fix useLinkClickHandler defaultShouldRevalidate default description#15322
brophdawg11 merged 1 commit into
remix-run:mainfrom
dfedoryshchev:docs/uselinkclickhandler-revalidate-default

Conversation

@dfedoryshchev

Copy link
Copy Markdown
Contributor

The useLinkClickHandler JSDoc documents defaultShouldRevalidate as "Defaults to true", but the hook applies no default: when the option is omitted it is passed through as undefined and the router falls back to its standard revalidation behavior, which is not unconditional revalidation.

Updated the @param description to describe the actual default behavior.

@github-actions

Copy link
Copy Markdown
Contributor

✅ CLA Signed

Thanks for signing the Contributor License Agreement.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ No Change File Found

This PR doesn't include a change file which is used for automated release notes.
If your change affects users, please add one (or more) change files and commit the generated file(s).

pnpm run changes:add

This script requires Node 24+. If you are on a lower version, please add a file manually

Not every PR needs a change file — you can skip this step if the change is internal-only
(tests, tooling, docs)

@brophdawg11 brophdawg11 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@brophdawg11 brophdawg11 merged commit 3d83ad4 into remix-run:main Jul 16, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants